Use env variables to set SMTP settings#222
Use env variables to set SMTP settings#222tapaswenipathak merged 2 commits intoOpenSourceHelpCommunity:predevfrom
Conversation
4c27cf1 to
3f4cd11
Compare
|
Closing this PR since we do not need email related features at the moment. #224 makes this PR obsolete |
| EMAIL_HOST_PASSWORD = os.getenv('EMAIL_PASSWORD', '') | ||
| EMAIL_PORT = os.getenv('EMAIL_PORT', '') | ||
| EMAIL_USE_TLS = True | ||
|
|
There was a problem hiding this comment.
Register/Login is of no good use if we don't confirm the email. Currently we have it hidden on master, once we are up with check we would make it live. This is needed and cannot be removed. Did you get the chance to set it on Heroku? or does this not work for you? looks good.
There was a problem hiding this comment.
@tapasweni-pathak There is a known issue with Gmail and Heroku. Heroku servers are located all around the world. When we try to send an email via Heroku server, Gmail blocks the request considering the unusual origin of the request. We could try and see if this works, but I'm not sure if this email verification via emails from gmail should be used on production.
|
@tapasweni-pathak Let's merge this and see if it works. We'll never know until we try ;) |
Fixes #204
Note - The application will return an Internal Server Error if any of these variables are not set.